-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Documentation for ONNX export of Megatron Models #6914
Documentation for ONNX export of Megatron Models #6914
Conversation
Signed-off-by: Asfiya Baig <[email protected]>
Signed-off-by: Asfiya Baig <[email protected]>
Could you add this to index.rst so it shows up in the navigation pane? |
@ericharper would the |
I think it's better at a higher level |
Signed-off-by: Asfiya Baig <[email protected]>
Added the change. Is that what you had in mind? |
That is a good change. But also link
|
Signed-off-by: Asfiya Baig <[email protected]>
Ah I see, just included it in index.rst as well |
Signed-off-by: Asfiya Baig <[email protected]>
for more information, see https://pre-commit.ci
Signed-off-by: Asfiya Baig <[email protected]>
Signed-off-by: Asfiya Baig <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you!
What does this PR do ?
Adds documentation for ONNX export of Megatron Models